Skip to content

Add FromIterator<HeaderName> to Vary header#223

Merged
seanmonstar merged 1 commit intohyperium:masterfrom
syphar:from-headername-vary
Mar 2, 2026
Merged

Add FromIterator<HeaderName> to Vary header#223
seanmonstar merged 1 commit intohyperium:masterfrom
syphar:from-headername-vary

Conversation

@syphar
Copy link
Contributor

@syphar syphar commented Feb 24, 2026

Resolves #125 ,
see also #210

( I like using typed headers, want to use it for docs.rs for a new feature :) )

@syphar
Copy link
Contributor Author

syphar commented Feb 24, 2026

seeing the MSRV test failure.

What solution do you prefer?

I can see that http sets 1.57: https://docs.rs/crate/http/latest/source/Cargo.toml.orig

@seanmonstar
Copy link
Member

Oh yea, it can be the same as http. Want to submit that in a separate PR?

@syphar
Copy link
Contributor Author

syphar commented Feb 24, 2026

Oh yea, it can be the same as http. Want to submit that in a separate PR?

#224

@syphar syphar force-pushed the from-headername-vary branch from b5dc79e to 98ed9dc Compare February 26, 2026 05:36
@syphar
Copy link
Contributor Author

syphar commented Feb 26, 2026

rebased this PR to fix CI

@seanmonstar seanmonstar merged commit e900f04 into hyperium:master Mar 2, 2026
6 checks passed
@syphar syphar deleted the from-headername-vary branch March 2, 2026 16:51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Support custom Vary header

2 participants